Skip to content

fix: exclude bot reviewers from Review Stacks#188

Merged
coji merged 3 commits intomainfrom
fix/exclude-bot-from-review-stacks
Mar 16, 2026
Merged

fix: exclude bot reviewers from Review Stacks#188
coji merged 3 commits intomainfrom
fix/exclude-bot-from-review-stacks

Conversation

@coji
Copy link
Owner

@coji coji commented Mar 16, 2026

Summary

  • Review Queue: getPendingReviewAssignments で Bot タイプのレビュアーを除外。copilot-pull-request-reviewer 等が表示されなくなる
  • Unassigned 判定: hasAnyReviewer サブクエリでも Bot を除外。Bot のみアサインされた PR は「レビュアー未アサイン」として Unassigned に表示される

Test plan

  • 既存テスト通過（aggregate-stacks.test.ts）
  • typecheck 通過
  • トレカセンターの Review Stacks で copilot-pull-request-reviewer が消えていることを確認
  • Bot のみアサインの PR が Unassigned に表示されることを確認

🤖 Generated with Claude Code

@coji @claude
fix: exclude bot reviewers from Review Stacks
66d416a 
getPendingReviewAssignments から Bot タイプのレビュアーを除外し、
Review Queue に copilot-pull-request-reviewer 等が表示されないようにした。

hasAnyReviewer サブクエリでも Bot を除外し、Bot のみアサインされた
PR は Unassigned として表示されるようにした。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 2026 •
edited
Loading

Warning

Rate limit exceeded

@coji has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 39 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 204af0ce-38e6-4701-b571-6d6df7fc07b4

📥 Commits

Reviewing files that changed from the base of the PR and between a520ede and 5c254e2.

📒 Files selected for processing (6)
  • app/libs/tenant-query.server.ts
  • app/routes/$orgSlug/analysis/reviews/+functions/queries.server.ts
  • app/routes/$orgSlug/throughput/deployed/+functions/queries.server.ts
  • app/routes/$orgSlug/throughput/merged/+functions/queries.server.ts
  • app/routes/$orgSlug/throughput/ongoing/+functions/queries.server.ts
  • app/routes/$orgSlug/workload/+functions/stacks.server.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/exclude-bot-from-review-stacks
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coji and others added 2 commits March 16, 2026 13:38
@coji @claude
perf: optimize hasAnyReviewer subquery to avoid per-row JOIN
1554f5c 
LEFT JOIN in EXISTS subquery を NOT IN subselect に置き換え。
Bot ログインリストは一度だけ評価され、per-row JOIN が不要になる。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coji @claude
refactor: extract excludeBots helper to eliminate Bot filter duplication
5c254e2 
7箇所に散在していた companyGithubUsers.type != 'Bot' フィルタパターンを
app/libs/tenant-query.server.ts の excludeBots ヘルパーに集約。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coji coji merged commit ba4504d into main Mar 16, 2026
6 checks passed
@coji coji deleted the fix/exclude-bot-from-review-stacks branch March 16, 2026 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@coji